Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rewrote CI to Pascal #138

Merged
merged 5 commits into from
Jan 16, 2025
Merged

Rewrote CI to Pascal #138

merged 5 commits into from
Jan 16, 2025

Conversation

theavege
Copy link
Contributor

@theavege theavege commented Jan 15, 2025

I would be interested in using CI on Pascal?
Would it be convenient for you to explain why zip archives are in the repository?

@peazip
Copy link
Owner

peazip commented Jan 15, 2025

peazip-sources/dev/metadarkstyle/ should not be deleted.
peazip-soirces should replicate the content of the standalone sources package, which needs metadarkstyle in order to successfully compile for Windows systems.

@theavege
Copy link
Contributor Author

peazip-soirces should replicate the content of the standalone sources package, which needs metadarkstyle in order to successfully compile for Windows systems.

But I made autodownload from lazarus package

@peazip
Copy link
Owner

peazip commented Jan 16, 2025

That is OK for the automated procedure, but at the same time I need to preserve the availability of metadarkstyle sources for manual building.

The subdirectory, and the standalone source package, contains a local copy of metadarkstyle, which is tested for working correctly with the current version of the source code of the application.

Ideally the local version should work as fallback if the online version of the package on Lazarus repository gets for any reason unavailable, corrupted by a supply chain attack, abandoned, or if an update makes it temporarily not fully working along the rest of the codebase.

@peazip
Copy link
Owner

peazip commented Jan 16, 2025

If it is OK for you, I can merge the pull request, and then restore the metadarkstyle subdirectory in peazip-sources.

@peazip peazip merged commit 6b8096a into peazip:sources Jan 16, 2025
2 checks passed
@theavege theavege deleted the upd/ci branch January 16, 2025 19:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants